x86emul: drop pointless and add useful default cases
authorJan Beulich <jbeulich@suse.com>
Tue, 5 Jul 2016 09:17:25 +0000 (11:17 +0200)
committerJan Beulich <jbeulich@suse.com>
Tue, 5 Jul 2016 09:17:25 +0000 (11:17 +0200)
commita800e4f6111842183f2476cd1d82a159fb5c92cb
treeeac9b7c2b9a0104d2f72a750446b0eaa1cd8727f
parentf86eabea5d0fe1815b58f21259edf317ff91a65a
x86emul: drop pointless and add useful default cases

There's no point in having default cases when all possible values have
respective case statements, or when there's just a "break" statement.

Otoh the two main switch() statements better get default cases added,
just to cover the case of someone altering one of the two lookup arrays
without suitably changing these switch statements.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
xen/arch/x86/x86_emulate/x86_emulate.c